Skip to content

Conversation

@NoelStephensUnity
Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity commented Jul 28, 2025

Purpose of this PR

This PR has two distributed authority related updates that is low risk and reduces the initial session owner to server synchronization down to one synchronization message.

Jira ticket

N/A

Changelog

  • Changed: The first session owner no longer sends two synchronization messages to the service.

Documentation

  • No tests have been added.
  • No documentation changes or additions were necessary.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

  • Manual testing done

Automated tests:

  • Covered by existing automated tests
  • Covered by new automated tests

Does the change require QA team to:

  • Review automated tests?
  • Execute manual tests?
  • Provide feedback about the PR?

Backports

No backport is needed since this is specific to distributed authority.

Session owner promotion to prepare for the potential of providing user-determined session owner promotion.
Removing the "double synch" from within the connection approval handler method as it is no longer needed.
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner July 28, 2025 15:14
@NoelStephensUnity NoelStephensUnity enabled auto-merge (squash) October 16, 2025 15:40
@NoelStephensUnity NoelStephensUnity merged commit bcb73cd into develop-2.0.0 Oct 16, 2025
24 checks passed
@NoelStephensUnity NoelStephensUnity deleted the chore/session-owner-promotion-and-connection-approved-upates branch October 16, 2025 17:22
@NoelStephensUnity NoelStephensUnity restored the chore/session-owner-promotion-and-connection-approved-upates branch October 16, 2025 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants